Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: intro install-dist module in root #2552

Merged
merged 12 commits into from
Jun 13, 2024
Merged

chore: intro install-dist module in root #2552

merged 12 commits into from
Jun 13, 2024

Conversation

VGalaxies
Copy link
Contributor

@VGalaxies VGalaxies commented Jun 5, 2024

  • intro install-dist module in root
  • move release-docs and scripts from hugegraph-server/hugegraph-dist to install-dist
  • change package artifact name to apache-hugegraph-incubating-pd/server-xxx

@VGalaxies VGalaxies self-assigned this Jun 5, 2024
Copy link

codecov bot commented Jun 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.98%. Comparing base (506850c) to head (3a230e5).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2552      +/-   ##
============================================
+ Coverage     56.94%   56.98%   +0.04%     
  Complexity      827      827              
============================================
  Files           612      612              
  Lines         49672    49672              
  Branches       6681     6681              
============================================
+ Hits          28284    28304      +20     
+ Misses        18572    18558      -14     
+ Partials       2816     2810       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

pom.xml Outdated
@@ -93,6 +93,7 @@
<module>hugegraph-server</module>
<module>hugegraph-pd</module>
<module>hugegraph-store</module>
<module>dist</module>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dist or hugegraph-dist 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The module name "hugegraph-dist" is currently taken... Consider renaming after "hugegraph-dist" is renamed to "server-dist".

@VGalaxies VGalaxies marked this pull request as ready for review June 7, 2024 11:23
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. ci-cd Build or deploy dependencies Incompatible dependencies of package labels Jun 7, 2024
.gitignore Outdated Show resolved Hide resolved
@VGalaxies VGalaxies changed the title chore: intro dist module in root chore: intro install-dist module in root Jun 10, 2024
Comment on lines -55 to +61
initial-store-list: 127.0.0.1:8502
initial-store-list: 127.0.0.1:8500

raft:
# The address of the local raft service
address: 127.0.0.1:8610
# The service address of the PD cluster
peers-list: 127.0.0.1:8610,127.0.0.1:8611,127.0.0.1:8612
peers-list: 127.0.0.1:8610
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any potential impact?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently, no side effects have been found. The modification is intended to use org.apache.hugegraph.pd.boot.HugePDServer to quickly start a single node PD.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 12, 2024
@liuxiaocs7 liuxiaocs7 merged commit 1e97521 into master Jun 13, 2024
16 checks passed
@liuxiaocs7 liuxiaocs7 deleted the intro-dist branch June 13, 2024 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-cd Build or deploy dependencies Incompatible dependencies of package lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants